css lookup: Replace g_return by gtk_internal return

These functions are called in an inner loop in
gtk_css_style_provider_lookup.

css selector: Use a builtin to count bits

css matcher: Avoid a redundant call

The sole caller passes an interned string, so we don't
need to call g_intern_string again.

css selector: Take advantage of interned strings

We know these strings are interned, no need to use strcmp or
g_str_hash for hem.

bitmask: Trivial formatting fix

bitmask: Optimize no-op resizes

Statistics for the gtk3-demo listbox example show that the
vast majority of calls to _gtk_allocated_bitmask_resize go
from a size of 2 to 2. Don't needlessly call realloc() in
this case.

bitmask: Optimize intersect

The functions was written in a way that would possibly
resize the mask twice, which is not necessary.

widget-factory: Simplify the ui file

gtk-demo: Simplify ui files

Run gtk-builder-tool simplify over all templates

gtk-builder-tool: Canonicalize booleans

Always write booleans as '0' or '1'.

builder: Minor tweak

widgetpath: Implement using GtkCssNodeDeclaration

This removes a bunch of code but more importantly allows providing the
declaration in CSS matchers.

cssnode: Move clear_classes() into the NodeDeclaration

The previous code was crashing when used as the returned classes array
would have been invalid after the first deletion. So if a 2nd class
would be deleted, invalid memory might have been referenced.

stylecontext: Fix list_classes() after optimization

Otherwise a junk value is returned.

widgetpath: Remove bad function prototypes

iconcache: Guard deprecated gdk-pixbuf APIs

cssnode: Optimize listing of classes

Avoid creating the GList of strings when the GQuarks
will just be determined again. Instead have
gtk_style_context_list_classes() do it specifically.

container: Don't add classes twice

_gtk_widget_create_path() already adds the style classes.

gtkplug.c: remove draw function, not needed

Background will be rendered by parent class - GtkWindow.

https://bugzilla.gnome.org/show_bug.cgi?id=754854

Improve printer distinction for option retrieval

In order to retrieve the user options for a printer, the respective
printer name is used.
This fixes the comparison of printer names to avoid that the options of
another printer are accidently read whose name starts with the same
letters, but is longer (e.g. "myprinterlongername" instead of
"myprinter").

This fixes Bug 753628.

HighContrast: make sidebar list rows taller

sync with Adwaita

https://bugzilla.gnome.org/show_bug.cgi?id=754866

GtkFileSystem: Add to the list of remote filesystems

Add afp and google-drive to the list of remote filesystems.

https://bugzilla.gnome.org/show_bug.cgi?id=754746

Updated Slovak translation

GtkWidget: Fix typo in #ifdef

It's ENABLE, not ENABEL. Also fix the corresponding #endif comments.

GtkContainer: Fix paramter name in doc comment

Optimize gtk_css_node_declaration_has_class

This function is getting called a lot. Statistics for the gtk3-demo
listbox example show most calls with 0-4 classes. Unrolling the cases
a bit brings the instruction count in callgrind from 93M to 52M.

More bit manipulation speedups

The same trick that was applied to _gtk_css_change_for_child in
the previous commit can be applied to _gtk_css_change_for_sibling
as well, and that is what this commit does.

With both functions converted, gtk_css_change_translate is no
longer needed and gets dropped.

Manipulate bits more efficiently

The function to translate GtkCssChange enum values to the PARENT
ones is called very frequently. This patch speeds it up tremendously.
The callgrind instruction count for this function in the listbox
demo goes from 108M to 7M.

match_all: Only allocate an array when needed

My statistics show that more than half of all calls end up
with 0 matches, so we can avoid some overhead by not allocating
an array at all in this case.

Handle match_all returning NULL

This is in preparation for not allocating an array at all if there
are no matches.

Optimize gtk_css_selector_match_all

We are dealing with really short lists here.
95% are < 10 matches, and the longest I've been able to record was 19.
So just do away with the hash table and do sorted insertion in
the array directly.

x11: Declare variable for the inner loop

Otherwise the outer loop control variable is messed up, and we end
up with uninitialized axes if there were any more valuators after
the XIKeyClass one.

This bug was sneakily introduced by fdb9a8e14, many thanks to
Carlos Soriano for helping spot the source of this bug.

https://bugzilla.gnome.org/show_bug.cgi?id=753431

Adwaita: special case selection mode lists checks

...otherwise the special selection mode 40x40px checks are used
shrinked to 16x16.
See https://bugzilla.gnome.org/show_bug.cgi?id=754610

Update the gtk3-demo man page

We have options now !

gtk-demo:Add a --list option

This just shows the available examples and then exists.

gtk-demo: Make --autoquit a simple option

No real need to specify the amount of delay, just make this
a plain boolean.

Use G_ENABLE_CONSISTENCY_CHECKS for template finalization checks

css selector: Use the quarks we have

Unless I'm missing something, there is no reason to go from
quarks back to strings to implement hasing or comparison.

style provider private: Use gtk_internal_return_if_fail

Clean up debug features

Introduce a GTK_DEBUG_CHECK() macro and use it to check for
GTK_DEBUG flags everywhere. Also guard all such places by

widget: Use G_ENABLE_CONSISTENCY_CHECKS for invariants checking

The logic here is that G_ENABLE_DEBUG is for compiling out
debug spew that can be triggered at runtime with the GTK_DEBUG
environment variable, while G_ENABLE_CONSISTENCY_CHECKS is for
consistency checks that are applied unconditionally.

size request: Redo the recursion checks

Use G_ENABLE_CONSISTENCY checks for the recursion checks
in gtksizerequest.c

Introduce G_ENABLE_CONSISTENCY_CHECKS

Use a separate G_ENABLE_CONSISTENCY_CHECKS define to guard internal
consistency checks that are applied unconditionally if they are enabled,
such as the widget invariants checking. Interactive debug spew that can
be triggered at runtime with the GTK_DEBUG environment variable is still
guarded by the G_ENABLE_DEBUG define.

The mapping from enable-debug levels to defines is as follows:

yes:     G_ENABLE_DEBUG G_ENABLE_CONSISTENCY_CHECKS
minimum: G_ENABLE_DEBUG G_DISABLE_CAST_CHECKS
no:      G_DISABLE_CAST_CHECKS G_DISABLE_ASSERT G_DISABLE_CHECKS

notebook: Drop use of G_DISABLE_CHECKS

Rewrite the one check that the G_DISABLE_CHECKS macro was used for
in terms of g_return_if_fail.

Drop an unused warning

It has been ifzeroed for many, many years.

Use a getter for widget direction

settings: Reuse a value

We've already gotten the value type, use it.

builder: Avoid some type-checking overhead

settings: Avoid a redundant check

g_value_transform already returns whether it was successful
or not, so no need to call g_value_type_transformable beforehand.

Updated po-properties French translation

Adwaita: set -GtkRange-trough-border: 0 for scrollbars

which fixes firefox scrollbar not being fittsy when maximixed.

HighContrast: remove edge highlight

https://bugzilla.gnome.org/show_bug.cgi?id=754551

widget path: Use gtk_internal_return_if_fail

css style property: Use gtk_internal_return_if_fail

css color: Use gtk_internal_return_if_fail

allocated bitmask: Use gtk_internal_return_if_fail

window: Use internal more internal getters

Trivial cleanup

widget: Use g_param_spec_get_name_quark

stack: Convert to gtk_container_child_notify_by_pspec

notebook: Convert to g_object_notify_by_pspec

grid: Convert to gtk_container_child_notify_by_pspec

For the same reasons as g_object_child_notify_by_pspec.

box: Convert to gtk_container_child_notify_by_pspec

For the same reasons as g_object_child_notify_by_pspec.

Add some more child property API

Add gtk_container_install_child_properties and
gtk_container_child_notify_by_pspec to mirror te corresponding
GObject APIs.

scrolled window: Convert to g_object_notify_by_pspec

app chooser button: Convert to g_object_notify_by_pspec

settings: Convert more fully to g_object_notify_by_pspec

It turns out that we always have the pspec at hand already.

Updated Norwegian bokmål translation.

builder: Simplify _gtk_builder_boolean_from_string

We can do this without calling strlen() or allocating memory.

builder: Move a function

Move _gtk_builder_boolean_from_string in the same source file
as the other _from_functions.

builder: Minor refactoring

Keep the GString in PropertyInfo around, instead of throwing it
away and just keeping the char*.

widget: Add more inline getters

Add a getter for state flags, and use it where appropriate.

settings: Avoid one case of g_object_notify

We already have the GParamSpec in hands, lets just use it instead
of looking it up again.

Avoid a string copy

We have an api to create a quark from a static string, lets use it.

treeview: Donate a priv variable to gtk_tree_view_init()

notebook: Don't needlessly cast in macros

That way we disable typechecks done by the compiler and won't notice
when we pass the wrong values to the macro. And that's bad.

notebook: Remove unused macros

stylecontext: Remove line that is a no-op

Creating the transient node will already copy the widget type. So
copying again will not do anything.

bitmask: Fix broken invert_range() implementation

The speed-up in 7da1f8a1ce145f48b6299fd8be86a64389ff0b0d was wrong in
certain conditions, even though it didn't trigger the existing
testsuite.

New testcase /bitmask/invert_range_hardcoded included.

Avoid an extra type check in gdk_screen_get_default

Just go through the vfunc directly.

Move GtkWidgetPrivate to gtkwidgetprivate.h

This lets us use inlined getters for members in there,
avoiding the type checks in the public getters.

Speed up gtk_allocated_bit_mask_invert_range

It was showing up on profiles and has a comment asking for speed.

Plug a leak

A type class reference gotten with g_type_class_ref must be released
with g_type_class_unref, or else we leak.

menu button: Convert to g_object_notify_by_pspec

This avoids pspec lookup overhead in g_object_notify.

container: Convert to g_object_notify_by_pspec

This avoids pspec lookup overhead in g_object_notify.

frame: Convert to g_object_notify_by_pspec

This avoids pspec lookup overhead in g_object_notify.

menu item: Convert to g_object_notify_by_pspec

This avoids pspec lookup overhead in g_object_notify.

text buffer: Convert to g_object_notify_by_pspec

This avoids pspec lookup overhead in g_object_notify.

text cell renderer: Convert to g_object_notify_by_pspec

This avoids pspec lookup overhead in g_object_notify.

builder: Convert to g_object_notify_by_pspec

This avoids pspec lookup overhead in g_object_notify.

gdk window: Convert to g_object_notify_by_pspec

This avoids pspec lookup overhead in g_object_notify.

device: Convert to g_object_notify_by_pspec

This avoids pspec lookup overhead in g_object_notify.

tree view column: Convert to g_object_notify_by_pspec

This avoids pspec lookup overhead in g_object_notify.

gesture single: Convert to g_object_notify_by_pspec

This avoids pspec lookup overhead in g_object_notify.

event controller: Convert to g_object_notify_by_pspec

This avoids pspec lookup overhead in g_object_notify.

radio button: Convert to g_object_notify_by_pspec

This avoids pspec lookup overhead in g_object_notify.

toggle button: Convert to g_object_notify_by_pspec

This avoids pspec lookup overhead in g_object_notify.

style context: Convert to g_object_notify_by_pspec

This avoids pspec lookup overhead in g_object_notify.

treeview: Convert to g_object_notify_by_pspec

This avoids pspec lookup overhead in g_object_notify.